Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add messageFallback property #44

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Conversation

spyric
Copy link

@spyric spyric commented May 8, 2017

This PR gives ability to notify application's owner about translation that is not exists in translation file.

@rmariuzzo rmariuzzo changed the base branch from master to develop May 8, 2017 21:09
@spyric
Copy link
Author

spyric commented May 20, 2017

Any change to get this merged soon?

@rmariuzzo rmariuzzo self-assigned this Jul 24, 2017
@rmariuzzo rmariuzzo self-requested a review July 24, 2017 17:38
@rmariuzzo
Copy link
Owner

Hey @spyric! I'm checking your proposal, is that a feature defined on Laravel's translator API ? The aim of this library is to provides API-compatible to laravel localization helper. :)

@spyric
Copy link
Author

spyric commented Jul 24, 2017

Hello @rmariuzzo . In fact it's not.

But Laravel gives ability to register custom translator and handle missed translations. Currently there is no way to handle missed translations that occured in JS files, so I offered optional thing that will give ability to track it.

@rmariuzzo
Copy link
Owner

rmariuzzo commented Jul 25, 2017 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants